Why is the alert about the cron-jobs commented out?

Also please rebase onto master to allow the checks to run.

We've just rebased against master, and it seems to have been applied cleanly.

Is it possible to re-examine merging this into master? It presently looks like all the checks are passing.

@vmw can you un-comment the cronjob alert please? I'm ok with merging it after that.

I just forced pushed an update to this branch that ensures the warning is re-enabled;

667c17a

Hmm, slight style issue: you've introduced several lines with white-space.

Please try to keep the code clean.

Hmm, slight style issue: you've introduced several lines with white-space.
Please try to keep the code clean.

You're likely right - I think there previously was a thread about having an offline linter.

I'm definitely happy to apply a linter if there is one, but, as a bit of a exception, it's difficult to automate this for us.

The commit itself fixes several bugs - we're looking to update to master, to determine the feasibility of our dedicating an engineer to help us make some more substantial changes.

Allowing the merging of these commits would help us determine the work associated with doing this from master - which is why I'm digging up these old requests.